Agility ticket

INT PR

#14885

Maria's Test Fix PR

#14969

Summary

~~Currently, if a customer creates an expense receipt during PPM closeout, but then deletes it before signing for the PPM closeout, the expense won't show up during document review on the SC side, but will show up on the AOA/SSW worksheet, which leads to incorrect payment amounts.

This branch adds a couple nil/status checks to ensure that these expenses are not added to the worksheet. The code now explicitly checks for an "APPROVED" status for all expenses.~~

Bug described above was actually fixed by a PR a day prior to this one being opened. After reviewing with other devs, this PR is instead intended to safeguard against a similar bug, where the if statement that was changed could still count deleted expenses if the status is somehow set back to NULL after going through the process (but not by deleting drafts as a customer, since this was fixed earlier)

Testing steps have been updated accordingly.

1. Create a PPM move as a customer, approve as SC and go to "Upload PPM Documentation" as Customer
2. Add expenses in the last section, they can be of any type, you'll want to make at least two. Keep track of the amounts of both/all of your expenses.
3. As SC, approve the PPM closeout.
4. As Customer, before you download your PPM packet, find the ID of one of your expenses in the moving_expenses table using either DBeaver or psql. Update its' status to NULL. Now download the payment packet, go to Page 2, and verify that the amounts in the "Claimed Expenses" section only show the amounts for the expense(s) you did not set to NULL.

Screenshots